Skip to content

Conversation

zirain
Copy link
Member

@zirain zirain commented Aug 11, 2025

@zirain zirain requested a review from a team as a code owner August 11, 2025 23:24
@istio-testing istio-testing added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Aug 11, 2025
@@ -149,9 +149,9 @@ parse_versions() {

step0() {
echo -e "\nStep 0: Check and prepare the environment\n"
parse_input "$1"
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

to prevent:

+ echo 'DRY_RUN: _gitpush origin master'
DRY_RUN: _gitpush origin master
+ gh repo set-default ''
expected the "[HOST/]OWNER/REPO" format, got ""
+ cleanup_env
+ popd

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

will gh repo accept user/istio.io.git as a repo format?

Copy link
Member

@dhawton dhawton Aug 12, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is already being done in #16770. And yes Craig, it will. Mentioned in the thread I was going to fix them in the master bump

@zirain zirain closed this Aug 12, 2025
@zirain zirain deleted the fix/create-minor-version branch August 12, 2025 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants